Skip to content

STAC-24722: move community stackpacks to contrib repo#1

Open
fvlankvelt wants to merge 9 commits into
mainfrom
STAC-24722-community-stackpacks-in-contrib
Open

STAC-24722: move community stackpacks to contrib repo#1
fvlankvelt wants to merge 9 commits into
mainfrom
STAC-24722-community-stackpacks-in-contrib

Conversation

@fvlankvelt

Copy link
Copy Markdown

No description provided.

@fvlankvelt fvlankvelt force-pushed the STAC-24722-community-stackpacks-in-contrib branch 2 times, most recently from 1b69118 to eb05292 Compare July 3, 2026 10:15
@fvlankvelt fvlankvelt force-pushed the STAC-24722-community-stackpacks-in-contrib branch from eb05292 to 7b16f4f Compare July 3, 2026 11:05
@fvlankvelt fvlankvelt force-pushed the STAC-24722-community-stackpacks-in-contrib branch 4 times, most recently from d604598 to 1f0f63d Compare July 3, 2026 14:11
@fvlankvelt fvlankvelt force-pushed the STAC-24722-community-stackpacks-in-contrib branch from 1f0f63d to 88d84ac Compare July 3, 2026 14:19
Comment thread .github/workflows/update-stackpacks-version-in-helm.yml
Comment thread .github/workflows/stackpacks-ci.yml
Comment thread .github/workflows/stackpacks-ci.yml Outdated
Comment thread .github/workflows/stackpacks-ci.yml
Comment thread .github/workflows/stackpacks-ci.yml
needs: [package]
if: |
always() &&
github.reposity == 'StackVista/contrib-stackpacks' &&

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo in reposity

build-and-push-docker-image:
name: Docker image build, scan, and push
needs: [package]
if: |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After the typo fix, I think redoing the fork fix would be helpful to verify - I don't think this gate would isolate fork PRs from same-repo branches. I suspect with pull_request triggers, github.repository is the base repository (StackVista/contrib-stackpacks), even for fork PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants